feat: remove MdxRoute and react-router-mdx dependency#1247
feat: remove MdxRoute and react-router-mdx dependency#1247jderochervlk wants to merge 1 commit intomasterfrom
Conversation
- Delete MdxRoute.res/.resi (all pages now handled by split routes) - Remove react-router-mdx from package.json and delete yarn patch - Remove react-router-mdx externals from Mdx.res - Remove mdxRoutes binding from ReactRouter.res - Remove react-router-mdx init from react-router.config.mjs and generate-route-types.mjs - Add DocsManualRoute for /docs/manual pages with sidebar and TOC - Add DocsReactRoute for /docs/react pages with sidebar and TOC - Add SyntaxLookupDetailRoute for /syntax-lookup/* detail pages - Add SidebarHelpers module for shared sidebar nav/category generation - Add MdxFile.loadAllAttributes to replace react-router-mdx loadAllMdx - Migrate BlogRoute to use MdxFile.loadAllAttributes - Migrate SyntaxLookupRoute to use MdxFile.loadAllAttributes - Migrate CommunityRoute to use SidebarHelpers and MdxFile.loadAllAttributes - Fix Mdx.sortSection comparator to handle all cases explicitly - Update routes.res to register all split routes and remove mdxRoutes Combines PRs #1221, #1222, #1225, #1227
Cloudflare deploymentDeployement ID: 22c9cbd5-7511-4348-9bf0-e797695298f2 ⛅️ wrangler 4.63.0 (update available 4.81.1) ✨ Uploading _redirects |
| @@ -0,0 +1,152 @@ | |||
| type loaderData = { | |||
There was a problem hiding this comment.
Isn't this type now duplicated? Is it too inflexible to reuse it over multiple routes?
|
|
||
| ## Pull Requests and Commits | ||
|
|
||
| - Use conventional commits format for commit messages (e.g. `feat: add new API docs`, `fix: resolve loader data issue`). |
There was a problem hiding this comment.
Uuuh, but this is not enforced yet, right? I know I will forget to do this often at least.
2 participants
Note
These changes were done with the assistance of AI
Combines PRs #1221, #1222, #1225, #1227